Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add skeletons for MantleBackup data synchronization #46

Merged
merged 12 commits into from
Oct 9, 2024

Conversation

ushitora-anqou
Copy link
Contributor

@ushitora-anqou ushitora-anqou commented Oct 3, 2024

No description provided.

@ushitora-anqou ushitora-anqou self-assigned this Oct 3, 2024
@ushitora-anqou ushitora-anqou force-pushed the implement-interface-of-mantle-primary branch from 77d281a to 3eb81bb Compare October 3, 2024 02:46
@ushitora-anqou ushitora-anqou changed the title check MB labels before any other write operations add skeletons for MantleBackup data synchronization Oct 3, 2024
@ushitora-anqou ushitora-anqou force-pushed the implement-interface-of-mantle-primary branch 3 times, most recently from 3d2cfbd to 78834f5 Compare October 3, 2024 04:38
@ushitora-anqou ushitora-anqou marked this pull request as ready for review October 3, 2024 06:35
internal/controller/mantlebackup_controller.go Outdated Show resolved Hide resolved
internal/controller/mantlebackup_controller.go Outdated Show resolved Hide resolved
internal/controller/mantlebackup_controller.go Outdated Show resolved Hide resolved
internal/controller/mantlebackup_controller.go Outdated Show resolved Hide resolved
internal/controller/mantlebackup_controller.go Outdated Show resolved Hide resolved
type dataSyncPrepareResult struct {
isIncremental bool
isSecondaryMantleBackupReadyToUse bool
diffFrom *mantlev1.MantleBackup // non-nil value iff isFull is false.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
diffFrom *mantlev1.MantleBackup // non-nil value iff isFull is false.
diffFrom *mantlev1.MantleBackup // non-nil value iff isIncremental is true.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed in ae497fd

internal/controller/mantlebackup_controller.go Outdated Show resolved Hide resolved
@ushitora-anqou
Copy link
Contributor Author

Note: I need to change the commit messages of some commits to reflect their fixed contents.

@ushitora-anqou ushitora-anqou force-pushed the implement-interface-of-mantle-primary branch from cbb4903 to 21c9c54 Compare October 8, 2024 02:36
@ushitora-anqou
Copy link
Contributor Author

@satoru-takeuchi I think I've fixed all the issues you mentioned. Could you review this PR again?

Note that I need to squash my fixup commits, so please don't merge it yet, even if the code looks good to you.

Copy link
Contributor

@satoru-takeuchi satoru-takeuchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please squash commits.

@ushitora-anqou ushitora-anqou force-pushed the implement-interface-of-mantle-primary branch from 2e874d5 to 416bad5 Compare October 9, 2024 02:38
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
@ushitora-anqou ushitora-anqou force-pushed the implement-interface-of-mantle-primary branch from 416bad5 to a44590a Compare October 9, 2024 02:41
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
@ushitora-anqou ushitora-anqou force-pushed the implement-interface-of-mantle-primary branch from a44590a to d399763 Compare October 9, 2024 02:46
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
…ess after calling it

Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
Signed-off-by: Ryotaro Banno <ryotaro.banno@gmail.com>
@ushitora-anqou ushitora-anqou force-pushed the implement-interface-of-mantle-primary branch from d399763 to 6d1d1e3 Compare October 9, 2024 02:49
@satoru-takeuchi satoru-takeuchi merged commit 4120d38 into main Oct 9, 2024
3 checks passed
@satoru-takeuchi satoru-takeuchi deleted the implement-interface-of-mantle-primary branch October 9, 2024 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants